home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c++-part2 / 18017 < prev    next >
Encoding:
Internet Message Format  |  1996-08-05  |  3.7 KB
  1. Path: news.sprintlink.net!datalytics!usenet
  2. From: Rob Stewart <stew@datalytics.com>
  3. Newsgroups: comp.lang.c++
  4. Subject: Re: Help! What's wrong with this?
  5. Date: Thu, 18 Apr 1996 11:32:54 -0400
  6. Organization: Datalytics, Inc
  7. Message-ID: <317660A6.56C7@datalytics.com>
  8. References: <4l22v3$1beo@useneta1.news.prodigy.com>
  9. NNTP-Posting-Host: 204.62.224.71
  10. Mime-Version: 1.0
  11. Content-Type: text/plain; charset=us-ascii
  12. Content-Transfer-Encoding: 7bit
  13. X-Mailer: Mozilla 2.0 (WinNT; I)
  14.  
  15. Brian Munroe wrote:
  16. > 
  17. > I'm trying to learn C++ and I've run into a strange problem in the
  18. > following code.
  19. > 
  20.  
  21. First, the code you submitted is C, not C++.  Just because 
  22. you're using a C++ compiler on it does not make it C++.  Let's 
  23. call it what it is.
  24.  
  25. > int const ADD_LEN = 40;
  26.  
  27. Try "const int ADD_LEN = 40" instead.
  28.  
  29. > int main()
  30. > {
  31. >   struct cust_list {
  32. >     char name[ADD_LEN];
  33. >     char add1[ADD_LEN];
  34. >     char add2[ADD_LEN];
  35. >     char add3[ADD_LEN];
  36. >   } customer;
  37. >   int flag;
  38. > 
  39. >   void get_customer_info(char temp[ADD_LEN], int flag);
  40.  
  41. This declaration should really come before main.
  42.  
  43. > 
  44. >   for (flag=0; flag<4; flag++)
  45. >   { switch (flag)
  46. >     { case 0: get_customer_info(customer.name, flag); break;
  47. >       case 1: get_customer_info(customer.add1, flag); break;
  48. >       case 2: get_customer_info(customer.add2, flag); break;
  49. >       case 3: get_customer_info(customer.add3, flag); break;
  50. >       default: exit(1);
  51. >     }
  52. >   }
  53.  
  54. This is the oddest thing I've ever seen.  Don't play games like 
  55. this, just write the following:
  56.  
  57.     get_customer_info(customer.name, 0);
  58.     get_customer_info(customer.add1, 1);
  59.     get_customer_info(customer.add2, 2);
  60.     get_customer_info(customer.add3, 3);
  61.  
  62. Having said that, it would be better for you to supply the 
  63. prompt string to get_customer_info rather than using a flag to 
  64. do it.  This allows the caller to more appopriately associate 
  65. the text of the prompt with the buffer being filled.
  66.  
  67. >   return 0;
  68. > }
  69. > 
  70. > void get_customer_info(char temp[ADD_LEN], int flag)
  71.  
  72. FYI: Declaring temp as char[ADD_LEN] doesn't guarrantee that you 
  73. get an array of that size.
  74.  
  75. > { int answer;
  76. >   int len;
  77. >   clrscr();
  78. >   switch (flag)
  79. >   { case 0:
  80. >      cout << "Enter the bidder's name in one of the following forms.\n";
  81. >      cout << "  Person: LAST NAME, FIRST NAME\n";
  82. >      cout << "  Business: COMPANY NAME\n\n";
  83. >      break;
  84. >     case 1:
  85. >      cout << "\nEnter first line of address\n";
  86. >      break;
  87. >     case 2:
  88. >      cout << "\nEnter second line of address\n";
  89. >      break;
  90. >     case 3:
  91. >      cout << "\nEnter third line of address\n";
  92. >      break;
  93. >     default:
  94. >      exit(1);
  95. >   }
  96. >   cin >> temp;
  97.  
  98. Try this instead.  It will eat whitespace before filling temp.
  99.  
  100.     cin >> ws >> temp;
  101.  
  102. >   len = strlen(temp);
  103. >   if (len < ADD_LEN)
  104. >    { strncat(temp, "                                        ",ADD_LEN-
  105. > len);}
  106. >   temp[ADD_LEN-1] = '\0';
  107.  
  108. Rather than have get_customer_info be dependent upon a global 
  109. value, why not pass the size of temp to it.  This reduces 
  110. dependence on the global to main, which is a better habit to 
  111. develop (later maintenance of code is easier with restricted 
  112. dependence on globals).
  113.  
  114. >   return;
  115. > }
  116. > 
  117. > When I run this all 4 prompts are outputted (Is that a word?), but only
  118. > input for the name and second line are  accepted. The input line doesn't
  119. > accept input for the first and third lines. I'm guessing that there is a
  120. > return character caught in the input buffer after I input the name and
  121. > second address lines and it's read in for the first and third lines. If
  122. > so, how do I get it out of the buffer and what the heck is it doing there
  123. > in the first place !?
  124. > 
  125. > Thanks for your help.
  126.  
  127. -- 
  128. Robert Stewart        | My opinions are usually my own.
  129. Datalytics, Inc.    | stew@datalytics.com
  130.